Skip to content

Implemented reading MFT tracks from AOD files#8580

Merged
shahor02 merged 1 commit intoAliceO2Group:devfrom
pnwkw:fwd_tracks
Apr 23, 2022
Merged

Implemented reading MFT tracks from AOD files#8580
shahor02 merged 1 commit intoAliceO2Group:devfrom
pnwkw:fwd_tracks

Conversation

@pnwkw
Copy link
Contributor

@pnwkw pnwkw commented Apr 13, 2022

No description provided.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually, also the directory on /home/epn/ where the jsons are stored is jsons.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed this to follow the default directory from which ED reads them:

"datafolder,d", bpo::value<decltype(this->mDataFolder)>()->default_value("./json"), "name of the data folder")(

In any case I think the best solution would be to extract this value to a header, so it is in sync in both ED and the AOD workflow.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The addTrackToEvent takes GlobalTrackID (track type a.k.a Source + index in the source container + optional 2 bits) as an argument, while detectorMapToGID returns just a Source. Since here you don't care about the index, you can rename the detectorMapToGID to detectorMapToGIDSource and use it as

addTrackToEvent(tr, GID{0, detectorMapToGIDSrc(track.detectorMap())}, trackTime, 0.);

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this is now fixed.

@shahor02 shahor02 merged commit b6e0538 into AliceO2Group:dev Apr 23, 2022
@pnwkw pnwkw deleted the fwd_tracks branch May 5, 2022 12:31
@pnwkw pnwkw restored the fwd_tracks branch May 5, 2022 12:31
@pnwkw pnwkw deleted the fwd_tracks branch May 5, 2022 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants